Skip to content

feat(code): use user GitHub integrations for cloud tasks#1951

Merged
tatoalo merged 3 commits intomainfrom
chore/cloud-agent/user-github-integrations
May 1, 2026
Merged

feat(code): use user GitHub integrations for cloud tasks#1951
tatoalo merged 3 commits intomainfrom
chore/cloud-agent/user-github-integrations

Conversation

@tatoalo
Copy link
Copy Markdown
Contributor

@tatoalo tatoalo commented Apr 30, 2026 •
edited
Loading

Problem

cloud task creation and handoff should use the user's linked GitHub identity instead of relying on a local gh token or the project gh integration. This keeps PR authorship tied to the user who starts the task and works for Code-created and handoff-created cloud runs

Before:
gh_authorship_sad

After:
gh_authorship_happy

@tatoalo tatoalo self-assigned this Apr 30, 2026
@tatoalo tatoalo marked this pull request as ready for review April 30, 2026 11:53
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 30, 2026 •
edited
Loading

Comments Outside Diff (1)

  1. apps/code/src/renderer/hooks/useIntegrations.ts, line 1051-1053 (link)

    P2 Superfluous useEffect — no-op on mount

    This effect resets requestedLimit to REPOSITORIES_PAGE_SIZE, but useState(REPOSITORIES_PAGE_SIZE) already initialises the state to that value. The effect fires once on mount and writes the same value that is already there, so it has no observable effect and can be removed.

    Prompt To Fix With AI 
    This is a comment left during a code review.
Path: apps/code/src/renderer/hooks/useIntegrations.ts
Line: 1051-1053

Comment:
**Superfluous `useEffect` — no-op on mount**

This effect resets `requestedLimit` to `REPOSITORIES_PAGE_SIZE`, but `useState(REPOSITORIES_PAGE_SIZE)` already initialises the state to that value. The effect fires once on mount and writes the same value that is already there, so it has no observable effect and can be removed.

How can I resolve this? If you propose a fix, please make it concise.
Prompt To Fix All With AI 
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
apps/code/src/renderer/hooks/useIntegrations.ts:1051-1053
**Superfluous `useEffect` — no-op on mount**

This effect resets `requestedLimit` to `REPOSITORIES_PAGE_SIZE`, but `useState(REPOSITORIES_PAGE_SIZE)` already initialises the state to that value. The effect fires once on mount and writes the same value that is already there, so it has no observable effect and can be removed.

Reviews (1): Last reviewed commit: "feat(code): use user github integrations..." | Re-trigger Greptile

@tatoalo tatoalo force-pushed the chore/cloud-agent/user-github-integrations branch from bacac97 to 4d65024 Compare April 30, 2026 13:10
@tatoalo tatoalo requested a review from a team April 30, 2026 16:43
@k11kirky k11kirky added the Create release This will trigger a new release label May 1, 2026
@tatoalo tatoalo merged commit c127528 into main May 1, 2026
15 checks passed
@tatoalo tatoalo deleted the chore/cloud-agent/user-github-integrations branch May 1, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joshsny joshsny joshsny approved these changes

Assignees

@tatoalo tatoalo

Labels

Create release This will trigger a new release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tatoalo @joshsny @k11kirky